-
-
Notifications
You must be signed in to change notification settings - Fork 34.7k
deps: update undici to 7.21.0 #61683
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
Review requested:
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #61683 +/- ##
==========================================
+ Coverage 89.75% 89.76% +0.01%
==========================================
Files 674 674
Lines 204416 204416
Branches 39285 39282 -3
==========================================
+ Hits 183472 183493 +21
+ Misses 13227 13221 -6
+ Partials 7717 7702 -15 🚀 New features to boost your workflow:
|
This comment was marked as outdated.
This comment was marked as outdated.
e5e1cf5 to
e7bce4b
Compare
|
The CI failures are caused by a bug in undici 7.20.0 where When Fix PR: nodejs/undici#4812 |
|
This is fixed in undici v7.21.0 |
mcollina
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
|
Landed in 9afa802 |
PR-URL: #61683 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Matteo Collina <matteo.collina@gmail.com> Reviewed-By: Filip Skokan <panva.ip@gmail.com> Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com> Reviewed-By: Matthew Aitken <maitken033380023@gmail.com>
This is an automated update of undici to 7.21.0.
Closes #61679